Skip to content

E-Distance by scverse pertpy into rapids_singlecell #454

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

E-Distance by scverse pertpy into rapids_singlecell #454

wants to merge 23 commits into from

Conversation

selmanozleyen
Copy link
Member

@selmanozleyen selmanozleyen commented Sep 9, 2025 •
edited
Loading

Solves #431

initial measurements

gpu w kernels
Time taken: 0.18562102317810059 seconds
old cpu
Time taken: 26.695605278015137 seconds

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

selmanozleyen and others added 16 commits September 9, 2025 23:38
@selmanozleyen
vibe codded version + the version I simplied. it works roughly
28cc06f
@selmanozleyen
changes
4df0982
@selmanozleyen
simplify kernels and create a separate file
33d3fc1
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
a8b016b 
for more information, see https://pre-commit.ci
@selmanozleyen
move to float64 for precision.
9323bfe
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
2150b0a 
for more information, see https://pre-commit.ci
@selmanozleyen
bootstrapping
7faef57
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
fb10db6 
for more information, see https://pre-commit.ci
@selmanozleyen
swich to f32
6b7d7b2
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3bbf06a 
for more information, see https://pre-commit.ci
@selmanozleyen
refactor
7a3c003
@selmanozleyen
push working state
ecfc5cf
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
5783261 
for more information, see https://pre-commit.ci
@selmanozleyen
add documentation and write a comparison script
0496af0
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
a7cf6c4 
for more information, see https://pre-commit.ci
@selmanozleyen
Merge branch 'main' into feature/pertpy-edistance
8df093f
@selmanozleyen selmanozleyen requested a review from Intron7 October 16, 2025 09:33
@selmanozleyen
Copy link
Member Author

@Intron7 if this looks good to you then I will remove the tmp_scripts folder. But so far its ready for merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Intron7 Intron7 Awaiting requested review from Intron7

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@selmanozleyen